Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #38 #37 #39

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Fix #38 #37 #39

wants to merge 5 commits into from

Conversation

devslava
Copy link

Fixed adding values programatically.
Also fixed possible bug when there is no values and ENTER OR DPAD_CENTER button was clicked ( onItemClicked() listener was called despite of this ).

Looks lile #37 happens sometimes despite of fix. After this commit I had it only once and was not able to reproduce again.

@devslava devslava changed the title Fix #38 #37 Fix #38 Feb 12, 2016
@devslava devslava changed the title Fix #38 Fix #38 #37 Feb 12, 2016
@blazsolar
Copy link
Owner

Hey,

thanks for your PR. I'll have to have a closer look before I merge it. In mean time can you add yourself to CONTRIBUTORS.md file.

@blazsolar blazsolar added this to the Version 2.0 milestone Feb 23, 2016
@nilsi
Copy link

nilsi commented Nov 19, 2018

Could we merge this one? Its a bug we are currently experiencing as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants